placessidebar: fix documentation

Use the same explanation of why GtkPlacesSidebar::show-connect-server
was deprecated and its replacements on getter and setter.

Also, fix a mistakenly deprecated function.

inspector: Show monitor information

This is useful to verify the monitor information returned by
other backends.

inspector: Show wayland display information

Don't call it "X display" when we show useful information under
Wayland as well.

wayland: Return a display name

Make gdk_screen_make_display_name() return a likely correct
name for the Wayland socket we are using.

x11: Make selection handling work across screens

When dealing with selection events, we might see windows from
other screens in the requestor field. The current x11 backend
code fails to wrap these in a foreign GdkWindow, since we
don't have the corresponding GdkScreen anymore. Work around
this by creating such 'foreign screens' on demand. We still
maintain the 1:1 relation between the display and the screen
returned by gdk_display_get_default_screen().

https://bugzilla.gnome.org/show_bug.cgi?id=721398

Update Catalan translation

Plug a memory leak

GtkCssProvider was using the wrong unref function for GtkCssKeyframe
objects. Funnily, this didn't crash at all, the refcount was at the
same place in the struct - it just didn't free all the memory, causing
valgrind to complain.

quartz app menu: add hidpi support for menu icons

Use the new cairo to NSImage converter function
to set the device scale.

Remove the pixbuf converter function as this was
the last user.

quartz dnd: add hidpi support for gtk_drag_set_icon_surface()

Instead of storing the dnd icon as a pixbuf store it
as a cairo surface and transfer the device scale
when converting it to a NSImage. In the pixbuf/stock/named
setters convert to a surface instead (no hidpi support yet)

quartz dnd: fix drag icon position relative to cursor position

The y axis is reversed on osx

quartz: disable system shadows if a GdkWindow shadow is set.

The shadow will be drawn in the wrong place in those cases so all we can
do is disable it. This fixes double shadows drawn around menus, popups
and tooltips.

https://bugzilla.gnome.org/show_bug.cgi?id=734984

placessidebar: show Computer item when not in Other Locations

Previous commits removed from places sidebar the code related
to displaying the Computer item, which should be shown when
the sidebar is not displaying the Other Locations item.

Add back the item when the sidebar is not in Other Locations'
mode.

quartz: Fix building

Broken by 6d04183c5d6 and 6ac07a2e85ea

file chooser: Note when searching remote locations

Inform the user that we won't search recursively when searching
in remote locations.

Avoid more GFile<>uri roundtrips

Make GtkSearchHit carry a GFile instead of an uri. Most of the
search engines already have the object around, and converting
to an uri and back is unnecessary extra work.

search engine: Don't crawl remote locations

This is slow, and causes too much network I/O.
So skip locations that look like they are remote.

Add a helper function for remote locations

Add a helper function that says whether a location should be
considered remote. To determine this, we look at the filesystem
type reported by gvfs, and say 'remote' for sftp, webdav, ftp,
nfs and cifs.

Actually use search results from the model

We didn't connect to the signals, so we never picked up
any search results from the model engine. Good thing, since
it was returning the wrong thing.

Port to GtkSearchHit

GtkSearchEngineModel was still trying to return uris as hits.
This is not working anymore, we are expected to return GtkSearchHit
structs.

Always pass a location into the query

Failure to do so was causing search to start from the home dir
when the current folder is not represented on the sidebar.

Store locations as GFile

It is a bit pointless to have the file chooser get a uri from an
existing GFile to put in the query, only to have some of the search
engines reconstruct a GFile from it.

Remove a redundant assignment

Updated Slovak translation

Updated Greek translation

combobox: Use ui file to construct internal button child

combobox: Create button on init() and destroy in destroy()

The button is the same in both menu and list mode, so there's no need to
destroy and recreate it.

combobox: Add scroll support in list mode

combobox: Remove unused argument

The visible check is always FALSE, so no need to keep the code around.

combobox: Remove the box from the button

... when in menu mode.

Now that we don't have a separator anymore, the box is not needed
anymore.

combobox: Remove the separator

It was a special case and it's not used by themes anymore.

Plus, in the CSS world, if a theme wants a separator, it can just
style the button's border to create one.

Remove a duplicate string

dialog-information was twice in the list of accessible labels
for icon names.

listbox: Fix shift-selection

Selecting a range of rows with Shift-click was accidentally broken
in e6e511eb460d8fcd. Make it work again.

quartz: Implement NO_TARGET drag result

This is necessary to make notebook tab DND work.

https://bugzilla.gnome.org/show_bug.cgi?id=752638

quartz: Check if a drag ends inside a known window

If a drag ends inside a known window, set the dest_window field
in the drag context. This information is needed to implemented
notebook tab dragging.

https://bugzilla.gnome.org/show_bug.cgi?id=752638

GtkMenu: Properly detach menu when the window is detached

Without properly cleaning up GtkMenu private attach state
(GtkMenuAttachData) when the attached widget is freed, we would end up
with an invalid pointer to a freed widget. Trying to detach from that
widget would cause a segmentation fault.

https://bugzilla.gnome.org/show_bug.cgi?id=752761

wayland: Fix initial fullscreen monitor support

-1 means that we have no specific preference for an initial
fullscreen monitor, and -1 is less than the number of monitors,
so we would end up accessing invalid memory. Prevent that.

https://bugzilla.gnome.org/show_bug.cgi?id=752875

Fix a warning

Use standard cursor names

This changes GTK+ to use gdk_cursor_new_from_name() with the
'standard' css names, instead of GdkCursorType.

https://bugzilla.gnome.org/show_bug.cgi?id=652085

win32: Support all css cursor names

Approximate some of the resize cursors with similar cursors.

broadway: fix use-after-free on read errors

Call chain:
 - input_data_cb
   - broadway_server_read_all_input_nonblocking (input)
     - broadway_input_free (input)
       (now input is invalid)
     attempt to use input->active -> use-after-free

Make broadway_server_read_all_input_nonblocking return a boolean, TRUE
if the input was valid, FALSE otherwise. This allows input_data_cb to
detect whether the input was gone or not.

https://bugzilla.gnome.org/show_bug.cgi?id=741685

Fix x11-less build

We don't use pangoft2 at all in libgdk, but we are using it in libgtk
both under X and Wayland.

Don't use libcanberra without X

libcanberra calls x11 backend functions, so we can't use it
when the x11 backend is disabled.

places view: Documentation fixups

Rewrite gtk_places_sidebar_set_show_enter_location docs, add
references to the relevant signals, remove mentions of
GtkPlacesView from the docs, since it is not public at this point.

https://bugzilla.gnome.org/show_bug.cgi?id=752633

Added api to set a window to fullscreen on a given monitor.

https://bugzilla.gnome.org/show_bug.cgi?id=752677

Updated Portuguese translation

Updated POTFILES.skip

gtk-demo: Expand the filter model example

Show normal filtering as well.

widget-factory: hook up callbacks to the tab close buttons

Just to show that buttons in notebook tabs work.

Fix a typo

https://bugzilla.gnome.org/show_bug.cgi?id=752810

GdkEvent: Turn out parameters optional

As pointed out by lazka in IRC, /me clearly out of phase after allow-none.

GdkEvent: Fix introspection annotations on gdk_event_get(_root)_coords

These functions respect NULLs being passed, so the return values are
(nullable).

GtkGesture: minor cleanup

We're not really using the coordinates in that function, we just
double check it's not an unrelated event.

MSVC Builds: Fix Introspection Build

The deprecated and a11y sources of GTK+ were spun off some time ago into
their own Makefile.inc, which is then included by gtk/Makefile.am.  Fix the
generation of file sources for the GTK+ .gir.

gtk/gtkplacessidebar.c: Fix Build on C89

Ensure variables are declared at the top of the block, and remove a ';;',
which C89 compilers don't like.

Updated Portuguese translation

Updated Portuguese translation

file chooser dialog: Better spacing

When the dialog is as narrow as possible, we still want some
space between the entry and the search button, so use margins
around the entry and label instead of box spacing.

file chooser: Fix gtk_file_chooser_set_show_hidden

We should not override an explicitly set show-hidden value
when loading the setting.

https://bugzilla.gnome.org/show_bug.cgi?id=610925

inspector: Rescan toplevels when they change

This ensures that windows appear in the inspectors tree when
they are created, and it prevents GTK_DEBUG=interactive from
coming up with an empty object tree.

https://bugzilla.gnome.org/show_bug.cgi?id=752664

inspector: Add a function to rescan

Currently, we only ever scanned the toplevel list and
widget tree once. That is not enough.

Another fix for unmapped window states

The previous fix was falling into the crack between
realized and mapped - we would apply the state when a window
is just realized, then unset the _initially flag, and then
when the window gets mapped, we'd undo the state. To fix
this, go back to the way things were when these flags were
first introduced.

https://bugzilla.gnome.org/show_bug.cgi?id=752765

GtkWidget: Fix set_font_map docs

A typo and non-matching parameter names.

gdk-win32: Send fake release key event for shift key

Windows does not send any release key event for one of the shift keys
when both shift keys were pressed together. This commit solves
the problem by sending the extra release key event for the shift key
which was released as first, when the other shift key is released.

Other modifiers (e.g. Ctrl, Alt) do not have this problem.

https://bugzilla.gnome.org/show_bug.cgi?id=751721

gtk3-demo: Don't use xalign on GtkCheckButton

quartz: remove maximized state when the window position/size is changed

So a window can be maximized/zoomed again after being moved away from
its maximized position. This makes the zoom button on non-CSD windows
work as before.

quartz: implement maximized/unmaximized

Instead of using the default zoom behaviour use the internal
maximized state for selecting our own zoom target. This makes
zooming work for CSD windows where for some reason the
given default zoom target is the current window frame itself
resulting in a shadowless window of the same size.

While this makes the zoom button behave a bit different as expected
it makes things more consistent with other platforms and fixes CSD
zooming.

quartz: make moving CSD windows right up to the screen's menu bar work.

The default implementation prevents the window + shadow to move above
the menu bar. Change it so only the window is considered.

file chooser: Make keynav in save mode smoother

When using the location popup to complete to a directory name
(with a trailing /), we should not punish the user by making the
Open button insensitive and preventing the Enter key from doing
the expected thing (switching to that directory).

https://bugzilla.gnome.org/show_bug.cgi?id=752708

Remove debug spew

file chooser entry: Special-case ., .. and ~

Make sure entering the three special strings ".", ".." or "~" in
the location entry works as expected. They already worked correctly
if you append a '/' to force them to be recognized as the 'folder'
part, but that should not be necessary.

https://bugzilla.gnome.org/show_bug.cgi?id=752707

GtkWindow: Rework the _initially variables

Go back to use these variables only for pre-mapped state changes.
Their use got muddied over the years, and it was hard to keep track
of what is acutal state, and what just a queued request.

Revert "window: fix initial shadow width for maximized/fullscreen windows"

This reverts commit 740bcf5fe0ff9dd25856e5d000faff80bcf44eea.

Revert "gtkwindow: Reset maximize / fullscreen_initially on state changes"

This reverts commit 862e0dab03e12c2fb1f6d99dd85c588dccc15dbc.

file chooser: Fix a crash

The save_widgets_create function was not safe against
being called multiple times in save mode, calling
gtk_file_chooser_set_create_folders was a way to trigger
this crash.

toolbutton: linkify properties

toolbutton: fix misleading docs

First only label-widget clashes with icon-name and not the label property. In
addtion fix the propertyname to use '-' instead of '_'.

gtk-demo: Add a filter model example

This shows computed columns.

fontchooserwidget: Refactor code so we actually optimize

Previously, we would pango_font_describe() every time the code ran and
we wouldn't ever hit the optimized quick exit.

The code now is a lot more complex because the
compute-actual-value-when-required-the-first-time approach is not
supported out of the box in GtkTreeModel (or GValue).

gtk3-demo: Don't set xalign of GtkCheckButton

It's deprecated and doesn't make a difference here.

GtkImage: Plug a memory leak

gtk-demo: Properly clean up timeouts in search entry demo

places sidebar: Don't leak shortcuts

Poined out by valgrind.

GtkSearchEngine: Be more robust in cancellation cases

When the search is cancelled, we may end up with a tracker
dbus reply coming in after the GtkSearchEngine object is
already gone, and bad things happen. Prevent this by
using g_signal_connect_object instead of g_signal_connect.

GtkQuery: Make API follow GTK+ conversions

We generally have const getters for strings, and the two users
of this API were promptly leaking the (unexpected) copy they
got from gtk_query_get_location and gtk_query_get_text.

places sidebar: Always chain up in finalize

The GtkSidebarRow class was forgetting to do this. The visible
symptom of this in valgrind is that you notice GObject data
being leaked.

places sidebar: Plug a memory leak

Updated Czech translation

Use versioned deprecations more seriously

I invadvertendly introduced a dependency on a recent GLib recently,
by cherry-picking a fix that used new GLib API. This commit will
help catching such errors before release, by using the versioned
deprecation machinery to turn such events into build-time
warnings.

places sidebar: Improve docs

Clarify the deprecation note for ::show-connect-to-server.
https://bugzilla.gnome.org/show_bug.cgi?id=752633

places sidebar: Improve the docs

Reword the docs of gtk_places_sidebar_set_drop_targets_visible().

places sidebar: Fix a copy-paste leftover

The ::populate-all nick and blurb where not referrering to this
property at all.
https://bugzilla.gnome.org/show_bug.cgi?id=752633

toolbutton: improve docs wrt stock deprecation

Docuemnt where to get the images from.

Updated Polish translation

Updated Hebrew translation

3.17.5

Fix distcheck

Makefile.decl does not work well with scripts that stay
in srcdir, so generate test-simplify from test-simplify.in,
just os that it ends up in builddir.

places view: Disconnect from volume monitor

Just unreffing the volume monitor object won't stop it from emitting
signals that we are connected to. Disconnect from the signals in
finalize.

reftests: Fix underlines in link-coloring reftest

Underlines now need to be manually specified in CSS since GTK got
text-decoration support.

treeview: Don't underallocate editing widgets that are too big

combobox: Unconditionally draw background and border